Add app version metadata to Project settings (fixes #167) #170
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Flamki
force-pushed
the
feat/settings-version-number
branch
from
ef839b2 to
f4bd802
Compare
Contributor
Author
|
Key points:
Happy to address any feedback! |
Bl3f
requested changes
Feb 12, 2026
Contributor
Bl3f
left a comment
Bl3f
left a comment
There was a problem hiding this comment.
LGTM, just before we merge can you rebase on origin/main and also remove the FASTAPI_URL/PORT changes, they have been done on main already!
Thank for this
…em route, Application + MCP settings cards)
Flamki
force-pushed
the
feat/settings-version-number
branch
from
bb2bae2 to
a839c40
Compare
Bl3f
reviewed
Feb 12, 2026
| </SettingsCard> | ||
|
|
||
| {isAdmin && ( | ||
| <SettingsCard title='Application'> |
Contributor
There was a problem hiding this comment.
Instead of using inputs can you make the design flatter? I know we use input for project parameters, but it's because we might make them editable, where here it does not makes sense to edit.
| -e BETTER_AUTH_URL=http://localhost:5005 \ | ||
| -v /path/to/your/nao-project:/app/project \ | ||
| -e NAO_DEFAULT_PROJECT_PATH=/app/project \ | ||
| getnao/nao:latest |
| @@ -0,0 +1,54 @@ | |||
| import fs from 'fs'; | |||
Contributor
There was a problem hiding this comment.
You have a few eslint warnings.
| const db = drizzle(process.env.DB_URI || '', { schema: pgSchema }); | ||
|
|
||
| describe('userTable', () => { | ||
| (process.env.DB_URI ? describe : describe.skip)('userTable', () => { |
Contributor
There was a problem hiding this comment.
Not sure why u need to change this.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #167
Summary
system.versionexposing:versioncommitbuildDateAPP_VERSIONAPP_COMMITAPP_BUILD_DATEFASTAPI_PORTacross backend/CLI/docs/compose/workflow.Why
As requested in #167, admins need to identify exactly which app build is running, including commit SHA in Docker deployments.
Testing
APP_VERSION=1.2.3APP_COMMIT=abc1234APP_BUILD_DATE=2026-02-11T00:00:00Z/api/auth/sign-up/email/api/trpc/project.getCurrent/api/trpc/system.versionwith auth cookie and got:{"version":"1.2.3","commit":"abc1234","buildDate":"2026-02-11T00:00:00Z"}Notes